-
Notifications
You must be signed in to change notification settings - Fork 773
Conversation
37807e6
to
8e23995
Compare
Codecov Report
@@ Coverage Diff @@
## master #963 +/- ##
=======================================
Coverage 46.48% 46.48%
=======================================
Files 116 116
Lines 6908 6908
=======================================
Hits 3211 3211
Misses 3440 3440
Partials 257 257
Continue to review full report at Codecov.
|
Could you please add a column in the table and stuff metrics type into it? |
8e23995
to
17f2aa8
Compare
Sure! Done. |
docs/user_guide/metrics.md
Outdated
- dragonfly_supernode_pieces_downloaded_size_bytes_total{} - total size of pieces downloaded from supernode in bytes. counter type. | ||
- dragonfly_supernode_gc_peers_total{} - total number of peers that have been garbage collected. counter type. | ||
- dragonfly_supernode_gc_tasks_total{} - total number of tasks that have been garbage collected. counter type. | ||
Name | Type | Description |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So embarrassed. 😢 And could you please add a column in the table and stuff labels into it?
docs/user_guide/metrics.md
Outdated
- dragonfly_supernode_gc_tasks_total{} - total number of tasks that have been garbage collected. counter type. | ||
Name | Type | Description | ||
:----------------------------------------------------------------------- | :-------- | :---------- | ||
dragonfly_supernode_build_info{version, revision, goversion, arch, os} | gauge | build and version information of supernode. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope we can separate the metric name and its labels. So please add another column called labels
docs/user_guide/metrics.md
Outdated
Name | Type | Description | ||
:----------------------------------------------------------------------- | :-------- | :---------- | ||
dragonfly_supernode_build_info{version, revision, goversion, arch, os} | gauge | build and version information of supernode. | ||
dragonfly_supernode_http_requests_total{code, handler, method} | counter | total number of http requests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please capitalize all these metrics description sentences.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Signed-off-by: SataQiu <qiushida@beyondcent.com>
0b49dca
to
0b9b2ac
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for your great work!
LGTM. Good job. THX. |
docs: update the metrics as a table
docs: update the metrics as a table
Signed-off-by: SataQiu qiushida@beyondcent.com
Ⅰ. Describe what this PR did
docs: update the metrics as a table
Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗?)
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews